Skip to content

Conversation

@mjnovice
Copy link
Contributor

@mjnovice mjnovice commented Jan 15, 2026

Development Package

  • Use uipath pack --nolock to get the latest dev build from this PR (requires version range).
  • Add this package as a dependency in your pyproject.toml:
[project]
dependencies = [
  # Exact version:
  "uipath==2.5.30.dev1011174146",

  # Any version from PR
  "uipath>=2.5.30.dev1011170000,<2.5.30.dev1011180000"
]

[[tool.uv.index]]
name = "testpypi"
url = "https://test.pypi.org/simple/"
publish-url = "https://test.pypi.org/legacy/"
explicit = true

[tool.uv.sources]
uipath = { index = "testpypi" }

[tool.uv]
override-dependencies = [
    "uipath>=2.5.30.dev1011170000,<2.5.30.dev1011180000",
]

@github-actions github-actions bot added test:uipath-langchain Triggers tests in the uipath-langchain-python repository test:uipath-llamaindex Triggers tests in the uipath-llamaindex-python repository labels Jan 15, 2026
@mjnovice mjnovice marked this pull request as ready for review January 15, 2026 02:13
cristipufu

This comment was marked as outdated.

@mjnovice mjnovice added the build:dev Create a dev build from the pr label Jan 16, 2026
@cristipufu cristipufu self-requested a review January 18, 2026 05:55
@mjnovice mjnovice force-pushed the mj/simulation-debug branch 3 times, most recently from bd28996 to 29c1016 Compare January 21, 2026 20:26
Copy link
Collaborator

@akshaylive akshaylive left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🚢

mjnovice and others added 8 commits January 22, 2026 08:56
The test_debug_with_simulation_file_sets_context test was failing on
Windows because load_simulation_config() uses Path.cwd() to locate
simulation.json, which wasn't reliably pointing to the isolated
filesystem directory in async contexts on Windows. Added a Path.cwd()
patch to explicitly ensure it returns the correct directory, consistent
with how the unit tests already handle this scenario.

Co-Authored-By: Claude Sonnet 4.5 <[email protected]>
Changed test_debug_with_simulation_file_sets_context to mock
load_simulation_config directly rather than trying to patch Path.cwd().
This approach is more robust and avoids filesystem-related issues in
async contexts on Windows. The test now directly returns a MockingContext
from load_simulation_config, ensuring consistent behavior across platforms.

Co-Authored-By: Claude Sonnet 4.5 <[email protected]>
- Added MockingStrategyType import and used enum instead of strings
- Added isinstance type assertions for union types
- Removed unused type: ignore comment in cli_dev.py

All mypy errors resolved, tests still pass.

Co-Authored-By: Claude Sonnet 4.5 <[email protected]>
The type ignore comment is needed in CI environment where uipath.dev
module doesn't have type stubs. Locally it may appear unused but it's
required for the CI pipeline to pass.

Co-Authored-By: Claude Sonnet 4.5 <[email protected]>
Moved the type: ignore[import-untyped] comment from the imported name
to the import statement line for proper mypy recognition in CI.

Note: This will show as "unused-ignore" locally where uipath.dev has
py.typed marker, but is required in CI where it doesn't.

Co-Authored-By: Claude Sonnet 4.5 <[email protected]>
Ruff detected unused variable assignments in three test functions.
Removed the assignments as the result value is not checked in these tests.

Co-Authored-By: Claude Sonnet 4.5 <[email protected]>
Applied ruff formatter to make code more compact and consistent
with project formatting standards.

Co-Authored-By: Claude Sonnet 4.5 <[email protected]>
@mjnovice mjnovice force-pushed the mj/simulation-debug branch from 778b5cc to 649249c Compare January 22, 2026 16:56
Bumped package version from 2.5.29 to 2.5.30 and updated uv.lock.

Co-Authored-By: Claude Sonnet 4.5 <[email protected]>
@mjnovice mjnovice merged commit a60a0c8 into main Jan 22, 2026
89 checks passed
@mjnovice mjnovice deleted the mj/simulation-debug branch January 22, 2026 17:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

build:dev Create a dev build from the pr test:uipath-langchain Triggers tests in the uipath-langchain-python repository test:uipath-llamaindex Triggers tests in the uipath-llamaindex-python repository

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants